refactor(core): BreadcrumbStore added by Reversean · Pull Request #175 · codex-team/hawk.javascript

Reversean · 2026-03-25T20:42:46Z

Motivation

Part of the broader effort to extract environment-agnostic logic into @hawk.so/core (#151). BreadcrumbManager was a concrete, browser-coupled class living in @hawk.so/javascript — Catcher depended directly on it, making breadcrumb support impossible to share or override in a non-browser runtime.

What changed and why

BreadcrumbStore is an interface extracted into @hawk.so/core. Catcher now depends on that interface rather than the browser implementation. BrowserBreadcrumbStore (the renamed BreadcrumbManager) is the browser-specific implementation that stays in @hawk.so/javascript.

Core defines the contract, concrete catchers wire in the environment-appropriate implementation. A future NodeCatcher can provide its own BreadcrumbStore without inheriting any browser code.

Changes

core: BreadcrumbStore interface added and exported
javascript: BreadcrumbManager renamed to BrowserBreadcrumbStore, now implements BreadcrumbStore; Catcher updated to depend on the interface

Copilot

Pull request overview

This PR refactors breadcrumb handling by introducing a shared BreadcrumbStore contract in @hawk.so/core and updating the JavaScript SDK to use a browser-specific implementation (BrowserBreadcrumbStore) instead of the previous BreadcrumbManager.

Changes:

Added BreadcrumbStore (plus related types) to @hawk.so/core and re-exported them from the core entrypoint.
Refactored the JavaScript SDK breadcrumbs implementation to BrowserBreadcrumbStore and updated Catcher to use the new store API (add/get/clear).
Updated JavaScript tests and types to align with the new names and API surface (and removed the local breadcrumbs-api.ts type).

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
packages/javascript/tests/catcher.user.test.ts	Updates singleton reset/import to `BrowserBreadcrumbStore`.
packages/javascript/tests/catcher.transport.test.ts	Updates singleton reset/import to `BrowserBreadcrumbStore`.
packages/javascript/tests/catcher.test.ts	Updates imports/reset and helper ordering.
packages/javascript/tests/catcher.global-handlers.test.ts	Updates singleton reset/import to `BrowserBreadcrumbStore`.
packages/javascript/tests/catcher.context.test.ts	Updates singleton reset/import to `BrowserBreadcrumbStore`.
packages/javascript/tests/catcher.breadcrumbs.test.ts	Updates singleton reset/import to `BrowserBreadcrumbStore`.
packages/javascript/tests/catcher.addons.test.ts	Updates singleton reset/import to `BrowserBreadcrumbStore`.
packages/javascript/tests/breadcrumbs.test.ts	Renames the suite and updates API calls to `add/get/clear`.
packages/javascript/src/types/index.ts	Re-exports breadcrumb store types from `@hawk.so/core`.
packages/javascript/src/types/breadcrumbs-api.ts	Removes the local `BreadcrumbsAPI` definition in favor of core types.
packages/javascript/src/catcher.ts	Switches from `BreadcrumbManager` to `BrowserBreadcrumbStore` and updates the public `breadcrumbs` getter typing.
packages/javascript/src/addons/breadcrumbs.ts	Introduces `BrowserBreadcrumbStore` implementing the core `BreadcrumbStore` contract and renames browser hint type.
packages/core/src/index.ts	Re-exports breadcrumb store types from the core package entrypoint.
packages/core/src/breadcrumbs/breadcrumb-store.ts	Adds the new shared breadcrumb store contract and associated types.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Dobrunia · 2026-04-01T18:02:42Z

don't forget to bump version

Reversean · 2026-04-01T18:29:38Z

don't forget to bump version

Thx for reminder.

…n added

- fix breadcrumbStore field TSDoc in catcher - destroy BrowserBreadcrumbStore instance before each tests

…ted in catcher event processing pipeline - MessageProcessor interface and MessageHint type - BrowserMessageProcessor, BreadcrumbsMessageProcessor, ConsoleCatcherMessageProcessor, DebugMessageProcessor - replaced inline addon logic with sequential MessageProcessor pipeline

neSpecc · 2026-04-09T18:56:42Z

  public init(options: BreadcrumbsOptions = {}): void {
    if (this.isInitialized) {
-      log('[BreadcrumbManager] init has already been called; breadcrumb configuration is global and subsequent init options are ignored.', 'warn');
+      log('[BrowserBreadcrumbStore] init has already been called; breadcrumb configuration is global and subsequent init options are ignored.', 'warn');


I don't think users should see such a log. It's better to remove it.

Log shortened.

- MessageHint renamed to ErrorSnapshot - minor code style and naming issues fixed

neSpecc · 2026-04-22T16:51:40Z

  public init(options: BreadcrumbsOptions = {}): void {
    if (this.isInitialized) {
-      log('[BreadcrumbManager] init has already been called; breadcrumb configuration is global and subsequent init options are ignored.', 'warn');
+      log('Breadcrumbs store already initialized', 'warn');


Please, remove this log. Users should not see our debug logs.

If init() is somehow called twice, then we should either:

throw exception (if it is not normal case)

hand it silently (if it is ok)

neSpecc · 2026-04-22T16:54:26Z

+      this.breadcrumbStore = BrowserBreadcrumbStore.getInstance();
+      this.breadcrumbStore.init(settings.breadcrumbs ?? {});
+      this.messageProcessors.push(new BreadcrumbsMessageProcessor());


can we initialize BrowserBreadcrumbStore inside the BreadcrumbsMessageProcessor to incapsulate breadcrumbs logic in there and do not pass breadcrumbs to all processors? and get rid of ErrorSnapshot term

neSpecc requested a review from Copilot March 31, 2026 16:33

Copilot started reviewing on behalf of neSpecc March 31, 2026 16:33 View session

Copilot AI reviewed Mar 31, 2026

View reviewed changes

neSpecc requested a review from Dobrunia April 1, 2026 17:54

Dobrunia approved these changes Apr 1, 2026

View reviewed changes

Reversean force-pushed the refactor/breadcrumb-store branch from 9890ddb to afc73fa Compare April 4, 2026 09:13

Reversean added 3 commits April 4, 2026 12:14

refactor(core): BreadcrumbStore abstraction and browser implementatio…

c1996e6

…n added

fix: fixed review comments

afc73fa

- fix breadcrumbStore field TSDoc in catcher - destroy BrowserBreadcrumbStore instance before each tests

neSpecc approved these changes Apr 9, 2026

View reviewed changes

Reversean added 5 commits April 14, 2026 21:44

fix: review fixes

c3ffcae

- MessageHint renamed to ErrorSnapshot - minor code style and naming issues fixed

feature: added messageProcessors parameter in HawkInitialSettings

ed1da69

test: messageProcessors in Catcher tests added

d30f102

refactor: removed EventRejectedError

9f6753b

refactor: already initialized breadcrumbs log shortened

703af25

neSpecc requested changes Apr 22, 2026

View reviewed changes

refactor: removed breadcrumbs init log

9a10955

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

refactor(core): BreadcrumbStore added#175

refactor(core): BreadcrumbStore added#175
Reversean wants to merge 9 commits intomasterfrom
refactor/breadcrumb-store

Reversean commented Mar 25, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Dobrunia commented Apr 1, 2026

Uh oh!

Reversean commented Apr 1, 2026

Uh oh!

neSpecc Apr 9, 2026

Uh oh!

Reversean Apr 22, 2026

Uh oh!

neSpecc Apr 22, 2026 •

edited

Loading

Uh oh!

neSpecc Apr 22, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

Reversean commented Mar 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Motivation

What changed and why

Changes

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Dobrunia commented Apr 1, 2026

Uh oh!

Reversean commented Apr 1, 2026

Uh oh!

neSpecc Apr 9, 2026

Choose a reason for hiding this comment

Uh oh!

Reversean Apr 22, 2026

Choose a reason for hiding this comment

Uh oh!

neSpecc Apr 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

neSpecc Apr 22, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Reversean commented Mar 25, 2026 •

edited

Loading

neSpecc Apr 22, 2026 •

edited

Loading